docs: reduce the README banner to the brand lock-up - #143
Merged
Merged
Conversation
The full-width nav strip carried the network badge, token badge, theme control and workspace link into a README where none of them mean anything, and at README width the whole row rendered small. Keep only the brand lock-up — the commit-ring mark, the wordmark, and its SETTLEMENT ENGINE tag — on the same rounded panel, and scale every length by one factor so the proportions stay exactly those of `.top-nav`. The banner is still generated, not drawn: the palette comes from tokens.css, the mark geometry from CommitRing.tsx, and both labels from App.tsx. The canvas is 381x112 and sized to its own content, so the tag clears the panel edge with the fallback fonts GitHub renders.
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
oneshot | 5fd1d57 | Sep 13 2026, 02:21 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to #140, which merged before this correction existed.
#140 put the whole
.top-navstrip into the README: the network badge, tokenbadge, theme control and workspace link travelled with it, none of which mean
anything outside the running app, and at README width the full-width row
rendered small. This reduces the banner to the brand lock-up alone — the
commit-ring mark, the wordmark, and its
SETTLEMENT ENGINEtag — on the samerounded panel.
Every length is now the corresponding
.top-navvalue scaled by one factor(
64 / 36, the mark's banner size over its nav size), so the proportions stayexactly those of the shipped nav while the lock-up reads at README width. The
canvas is 381x112 and sized to its own content rather than fixed.
The banner is still generated, never drawn: the palette comes from
packages/brand/src/tokens.css, the mark geometry frompackages/brand/src/CommitRing.tsx, and both labels fromapps/web/src/App.tsx.tokens.cssremains the only source of a brand colour.Scope and acceptance criteria
Acceptance criteria:
README column.
.top-nav; no length is retuned by hand.three from front-end source.
markdownlint,prettier --check, andeslintstay green.Product and security invariants
Invariant notes:
Documentation and one build-time renderer only. No runtime code, contract,
migration, or configuration is touched, so the isolation, authorization, and
cap invariants are unchanged by construction.
Validation
Commands and results:
Independent review evidence
Gate A — exact candidate tree before push
Base commit SHA:
9177bbee4aa3705eae2038877f8183614ca89364Reviewer tool: not run
Verdict: not run
Findings or residual risks: Gate A was waived by the user for the README work
in docs: refresh README and add front-end derived banner #140; this follow-up carries the same waiver.
The reviewed tree equals the committed tree.
Gate B — exact remote PR head
Pull request URL/number: this PR
Remote head commit SHA:
5fd1d57Reviewer tool: not run
Verdict: not run
Findings or residual risks: waived with Gate A.
Gate B reviewed the current remote head and matches Gate A's approved tree, or a fresh Gate A was run for the changed tree.
Agent policy / repository-policyand all applicable CI checks pass.Risk and rollback
static build output, so a brand token, mark, or label change needs
scripts/render-nav-panel.mjsre-run or the README drifts from the UI again.Pill and panel widths come from per-family advance estimates, since Rubik
cannot be measured without a font engine.
the wide strip merged in docs: refresh README and add front-end derived banner #140 and no runtime behaviour changes.
Human merge
Draft: opened as a draft with Gate A and Gate B waived, matching how #140
was opened.